Deduplicate routine actions and badges across project routes

- Share edit/complete/archive form actions in routine-actions.ts; route
  servers become thin wrappers (strictest guard semantics in one place)
- Move parseFrequencyDays into validation.ts
- Extract ScoreBadge/StatusBadge components; use on project pages and
  the home page instead of duplicated scoreClass/statusClass helpers
- Fix archiveProject calling the routine archive with the project id

1 files changed, 11 insertions, 58 deletions

diff --git a/src/routes/projects/[projectId]/routine/[routineId]/+page.server.ts b/src/routes/projects/[projectId]/routine/[routineId]/+page.server.ts
index f2b6551..df4c2cc 100644
--- a/src/routes/projects/[projectId]/routine/[routineId]/+page.server.ts
+++ b/src/routes/projects/[projectId]/routine/[routineId]/+page.server.ts
@@ -1,26 +1,14 @@
-import { error, fail } from '@sveltejs/kit';
+import { error } from '@sveltejs/kit';
 import { getById } from '$lib/server/services/projects';
+import { loadCurrent, loadCompletions } from '$lib/server/services/routines';
 import {
-	edit,
-	complete,
-	loadCurrent,
-	loadCompletions,
-} from '$lib/server/services/routines';
-import { numericId, parseFormString } from '$lib/server/services/validation';
-import {
-	parseIntervalToDays,
-	computeRoutineState,
-} from '$lib/shared/routine-metrics';
+	completeRoutineAction,
+	editRoutineAction,
+} from '$lib/server/services/routine-actions';
+import { numericId } from '$lib/server/services/validation';
+import { computeRoutineState } from '$lib/shared/routine-metrics';
 import type { Actions, PageServerLoad } from './$types';
 
-function parseFrequencyDays(value: FormDataEntryValue | null): number {
-	const days = parseIntervalToDays(value?.toString() ?? '');
-	if (days === null) {
-		throw new Error('Invalid interval');
-	}
-	return days;
-}
-
 export const load: PageServerLoad = async ({ params }) => {
 	const projectId = numericId(params.projectId);
 	const routineId = numericId(params.routineId);
@@ -49,49 +37,14 @@ export const load: PageServerLoad = async ({ params }) => {
 
 export const actions: Actions = {
 	edit: async ({ request, params }) => {
-		const current = await loadCurrent(numericId(params.routineId));
-		if (!current) {
-			return fail(400, { error: 'Routine introuvable' });
-		}
-		if (current.archived) {
-			return fail(400, { error: 'Routine archivée' });
-		}
 		const data = await request.formData();
-		const name = parseFormString(data.get('name'));
-		if (!name) {
-			return fail(400, { error: 'Le nom est requis' });
-		}
-		const description = data.get('description')?.toString() ?? '';
-		try {
-			await edit(numericId(params.routineId), {
-				name,
-				frequencyDays: parseFrequencyDays(data.get('frequencyDays')),
-				description,
-			});
-		} catch {
-			return fail(400, { error: 'Fréquence invalide' });
-		}
-		return { success: true, type: 'edit' as const };
+		return editRoutineAction(data, numericId(params.routineId), {
+			type: 'edit',
+		});
 	},
 
 	complete: async ({ request, params }) => {
-		const current = await loadCurrent(numericId(params.routineId));
-		if (!current) {
-			return fail(400, { error: 'Routine introuvable' });
-		}
-		if (current.archived) {
-			return fail(400, { error: 'Routine archivée' });
-		}
 		const data = await request.formData();
-		try {
-			await complete(
-				numericId(params.routineId),
-				parseFormString(data.get('note')),
-				new Date().toISOString(),
-			);
-		} catch {
-			return fail(400, { error: 'Routine introuvable' });
-		}
-		return { success: true };
+		return completeRoutineAction(data, numericId(params.routineId));
 	},
 };